Skip to content

fix e2ee defaults masked in release builds#126

Merged
stephen-derosa merged 2 commits into
livekit:mainfrom
stephen-derosa:sderosa/bugfix/e2ee-defaults
May 12, 2026
Merged

fix e2ee defaults masked in release builds#126
stephen-derosa merged 2 commits into
livekit:mainfrom
stephen-derosa:sderosa/bugfix/e2ee-defaults

Conversation

@stephen-derosa
Copy link
Copy Markdown
Collaborator

@stephen-derosa stephen-derosa commented May 12, 2026

Overview

correctly populate defaults for e2ee proto

Testing

debug integration tests running locally

Linear Ticket

https://linear.app/livekit/issue/BOT-357/e2ee-missing-proto-fields

Comment thread src/ffi_client.cpp Outdated
float up missing e2ee fields

e2ee: toProto
@stephen-derosa stephen-derosa force-pushed the sderosa/bugfix/e2ee-defaults branch from 0c8d9f1 to 438394c Compare May 12, 2026 21:01
Copy link
Copy Markdown
Collaborator

@xianshijing-lk xianshijing-lk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one nit and one question.

lgtm

Comment thread include/livekit/e2ee.h
Comment thread include/livekit/e2ee.h
@stephen-derosa stephen-derosa force-pushed the sderosa/bugfix/e2ee-defaults branch from 8f11eb6 to add61d6 Compare May 12, 2026 22:01
@stephen-derosa stephen-derosa merged commit b682845 into livekit:main May 12, 2026
25 of 26 checks passed
@stephen-derosa stephen-derosa deleted the sderosa/bugfix/e2ee-defaults branch May 12, 2026 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants